Skip to content

feat: Add Executor support to Monitor for concurrent search (issue #16198) - #16433

Open
cyforkk wants to merge 2 commits into
apache:mainfrom
cyforkk:monitor-concurrency
Open

feat: Add Executor support to Monitor for concurrent search (issue #16198)#16433
cyforkk wants to merge 2 commits into
apache:mainfrom
cyforkk:monitor-concurrency

Conversation

@cyforkk

@cyforkk cyforkk commented Jul 29, 2026

Copy link
Copy Markdown

Summary

This PR adds support for passing an Executor to the Monitor for inter-segment concurrency in search operations.

Changes

  • Added Executor field to MonitorConfiguration with getter/setter methods
  • Modified Monitor constructor to store the Executor from configuration
  • Updated match() and debug() methods to use the Executor when creating IndexSearcher instances

Usage

Users can now configure concurrent search in Monitor:

MonitorConfiguration config = new MonitorConfiguration();
config.setExecutor(executor);
Monitor monitor = new Monitor(analyzer, config);

Related

Fixes #16198

🤖 Generated with Claude Code

…ache#16198)

Allow users to pass an Executor to MonitorConfiguration for inter-segment
concurrency in search operations. When set, the Monitor will use this
Executor when creating IndexSearcher instances.

Changes:
- Add Executor field to MonitorConfiguration with getter/setter
- Store Executor in Monitor constructor
- Use Executor when creating IndexSearcher in match() and debug() methods

Co-Authored-By: Claude <noreply@anthropic.com>
@cyforkk
cyforkk force-pushed the monitor-concurrency branch from 27b5780 to 030095c Compare July 29, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Monitor to support search concurrency

1 participant